As long as you're fixing this, I think it should say 2...36 instead of 2..36. The latter has a lot of precedent for meaning [2,36) instead of [2,36].

Two dots are used for inclusive range in match and in a couple comments elsewhere. Three dots are not used anywhere in rust. Wouldn't it be better to be consistent with those?

Oh it is? Guess I've never actually used the range functionality of match. I suppose remaining consistent with those is correct, though perhaps we should reevaluate that syntax.